Skip to content

BDMS-456#360

Merged
jirhiker merged 1 commit into
stagingfrom
BDMS-456-Migrate-WeatherData
Jan 9, 2026
Merged

BDMS-456#360
jirhiker merged 1 commit into
stagingfrom
BDMS-456-Migrate-WeatherData

Conversation

@jirhiker

@jirhiker jirhiker commented Jan 9, 2026

Copy link
Copy Markdown
Member

Why

This PR addresses the following problem / context:

  • Add legacy WeatherData support (model, migration, backfill, tests)
  • Ensure WeatherData backfill can upsert cleanly with a PK even if table pre-exists

How

Implementation summary - the following was changed / added / removed:

  • Added NMA_WeatherData model and corresponding unit tests
  • Added Alembic migration to create the WeatherData table and enforce the OBJECTID PK if the table
    already exists
  • Added WeatherData backfill script and wired it into the backfill orchestrator with
    BACKFILL_WEATHER_DATA

Notes

Any special considerations, workarounds, or follow-up work to note?

  • WeatherData backfill uses ON CONFLICT (OBJECTID) and depends on the PK constraint to exist
  • Set BACKFILL_WEATHER_DATA=false to skip in environments without the CSV loaded

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@jirhiker jirhiker merged commit 0d19770 into staging Jan 9, 2026
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant